Skip to content
This repository was archived by the owner on Oct 8, 2025. It is now read-only.

Add info about the new --control-* options #112

Merged
JTorreG merged 65 commits into from
Feb 27, 2024
Merged

Add info about the new --control-* options #112

JTorreG merged 65 commits into from
Feb 27, 2024

Conversation

@ac000
Copy link
Member

@ac000 ac000 commented Feb 26, 2024

This documents the new (in 1.32.0) --control-mode, --control-user & --control-group options to untid(8).

JTorreG and others added 30 commits February 22, 2024 11:58
@JTorreG
modify version and date in config
c241ce7
@JTorreG
add news document
6261d93
@JTorreG
Add changes to configuration doc
adb97c4
@JTorreG
update building from source
f619288
@JTorreG
update app languages and optional
14bda11
@JTorreG
moving content to the right sections
e41ac7c
@JTorreG
add configuration wasm section
34758c3
@JTorreG
update configuration steps
9f642ac
@JTorreG
fix trailing whitespace
230b213
@JTorreG
add samples changes
0b02b5b
@JTorreG
add conditional logging to config
91a163b
@JTorreG
make access log sub-section
c66f677
@JTorreG
add variable
c73b16a
@JTorreG
add hyperlink
4082e9c
@JTorreG
update language modules
44f5755
@JTorreG
add if to NJS places
d5eaa3e
@JTorreG @travisamartin
Apply suggestions from code review
f6c9030 
Co-authored-by: Travis Martin <33876974+travisamartin@users.noreply.github.com>
@JTorreG @travisamartin
Apply suggestions from code review
1f671fd 
Co-authored-by: Travis Martin <33876974+travisamartin@users.noreply.github.com>
@JTorreG @travisamartin
Apply suggestions from code review
1b82374 
Co-authored-by: Travis Martin <33876974+travisamartin@users.noreply.github.com>
@JTorreG
update line length
7957520
@JTorreG
Apply suggestions from code review
b80dfcc
@JTorreG @travisamartin
Apply suggestions from code review
0939a4b 
Co-authored-by: Travis Martin <33876974+travisamartin@users.noreply.github.com>
@JTorreG
Apply suggestions from code review
ebae145 
Adding suggestions from ac000
@JTorreG
Update source/howto/source.rst
c5bc960 
as per suggestions
@JTorreG @ac000
Update source/howto/samples.rst
d9e2472 
Co-authored-by: Andrew Clayton <a.clayton@nginx.com>
@JTorreG @ac000
Update source/howto/samples.rst
2b090b2 
Co-authored-by: Andrew Clayton <a.clayton@nginx.com>
@JTorreG
Update source/news/2024/unit-1.32.0-released.rst
151fb1d 
Co-authored-by: Timo Stark <t.stark@f5.com>
@JTorreG
Update source/news/2024/unit-1.32.0-released.rst
6f7ce9c 
Co-authored-by: Timo Stark <t.stark@f5.com>
@JTorreG @lcrilly
Update source/howto/source.rst
f21a06b 
Co-authored-by: Liam Crilly <liam.crilly@nginx.com>
@JTorreG
re-add removed section scripting
d5d75cd
@ac000 ac000 linked an issue Feb 26, 2024 that may be closed by this pull request
Add Control-Socket permission Options to HowTo - Startup / Shutdown #107
Closed
@netlify
Copy link

netlify bot commented Feb 26, 2024 •
edited
Loading

✅ Deploy Preview for nginx-unit ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit ab6c9a5
🔍 Latest deploy log https://app.netlify.com/sites/nginx-unit/deploys/65ddea84bf7eee0007aec569
😎 Deploy Preview https://deploy-preview-112--nginx-unit.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@ac000 ac000 requested a review from tippexs February 26, 2024 23:19
tippexs
tippexs reviewed Feb 27, 2024
View reviewed changes
lcrilly
lcrilly reviewed Feb 27, 2024
View reviewed changes
lcrilly
lcrilly reviewed Feb 27, 2024
View reviewed changes
lcrilly
lcrilly reviewed Feb 27, 2024
View reviewed changes
@ac000
Copy link
Member Author

ac000 commented Feb 27, 2024

Don't hyphenate UNIX-domain

$ git range-diff db855ae...d9cd0ec
1:  db855ae ! 1:  d9cd0ec howto/source.rst: Add info about the new --control-* options
    @@ source/howto/source.rst: counterparts, see :ref:`here <source-config-src>`.
                # unitd --control :nxt_hint:`unix:/path/to/control.unit.sock <Note the unix: prefix>`
      
     +   * - **--control-mode**
    -+     - Sets the permission of the UNIX-domain control socket. Default: 0600
    ++     - Sets the permission of the UNIX domain control socket. Default: 0600
     +
     +   * - **--control-user**
    -+     - Sets the owner of the UNIX-domain control socket.
    ++     - Sets the owner of the UNIX domain control socket.
     +
     +   * - **--control-group**
    -+     - Sets the group of the UNIX-domain control socket.
    ++     - Sets the group of the UNIX domain control socket.
     +
         * - **--group name**, **--user name**
           - Group name and user name used to run Unit's non-privileged

@ac000
Copy link
Member Author

ac000 commented Feb 27, 2024

Rebased onto release-1.32

$ git range-diff d9cd0ec...036920b
 1:  fd3afb4 <  -:  ------- WebAssembly is an 'internal' language module
 2:  d4605e6 <  -:  ------- fix issue-105
 -:  ------- >  1:  c241ce7 modify version and date in config
 -:  ------- >  2:  6261d93 add news document
 -:  ------- >  3:  adb97c4 Add changes to configuration doc
 -:  ------- >  4:  f619288 update building from source
 -:  ------- >  5:  14bda11 update app languages and optional
 -:  ------- >  6:  e41ac7c moving content to the right sections
 -:  ------- >  7:  34758c3 add configuration wasm section
 -:  ------- >  8:  9f642ac update configuration steps
 -:  ------- >  9:  230b213 fix trailing whitespace
 -:  ------- > 10:  0b02b5b add samples changes
 -:  ------- > 11:  91a163b add conditional logging to config
 -:  ------- > 12:  c66f677 make access log sub-section
 -:  ------- > 13:  c73b16a add variable
 -:  ------- > 14:  4082e9c add hyperlink
 -:  ------- > 15:  44f5755 update  language modules
 -:  ------- > 16:  d5eaa3e add if to NJS places
 -:  ------- > 17:  f6c9030 Apply suggestions from code review
 -:  ------- > 18:  1f671fd Apply suggestions from code review
 -:  ------- > 19:  1b82374 Apply suggestions from code review
 -:  ------- > 20:  7957520 update line length
 -:  ------- > 21:  b80dfcc Apply suggestions from code review
 -:  ------- > 22:  0939a4b Apply suggestions from code review
 3:  d9cd0ec = 23:  036920b howto/source.rst: Add info about the new --control-* options

@ac000
howto/source.rst: Add info about the new --control-* options
ab6c9a5 
This documents the new (in 1.32.0) --control-mode, --control-user &
--control-group options to untid(8).

Closes: nginx#107
Signed-off-by: Andrew Clayton <a.clayton@nginx.com>
@ac000
Copy link
Member Author

ac000 commented Feb 27, 2024

Rebase with latest release-1.32

$ git range-diff 036920b...ab6c9a5
 -:  ------- >  1:  ebae145 Apply suggestions from code review
 -:  ------- >  2:  c5bc960 Update source/howto/source.rst
 -:  ------- >  3:  d9e2472 Update source/howto/samples.rst
 -:  ------- >  4:  2b090b2 Update source/howto/samples.rst
 -:  ------- >  5:  fd3afb4 WebAssembly is an 'internal' language module
 -:  ------- >  6:  151fb1d Update source/news/2024/unit-1.32.0-released.rst
 -:  ------- >  7:  6f7ce9c Update source/news/2024/unit-1.32.0-released.rst
 -:  ------- >  8:  f21a06b Update source/howto/source.rst
 -:  ------- >  9:  d5d75cd re-add removed section scripting
 -:  ------- > 10:  9329fe9 cherrypick  PR106
 -:  ------- > 11:  629ca81 news scripting support rewrite
 -:  ------- > 12:  9c12ff6 add CLI enhancements to news
 -:  ------- > 13:  c234e47 Update source/scripting.rst
 -:  ------- > 14:  fa86bc4 Update source/scripting.rst
 -:  ------- > 15:  001618c Update source/scripting.rst
 -:  ------- > 16:  971eb7d Update source/howto/samples.rst
 -:  ------- > 17:  0dde02d fix reference
 -:  ------- > 18:  47c4862 update deprecated warning
 -:  ------- > 19:  9199fe6 Update source/howto/source.rst
 -:  ------- > 20:  3da93db add Cache-control example
 -:  ------- > 21:  910dba3 add vars to njs objects
 -:  ------- > 22:  149e599 initial commit
 -:  ------- > 23:  2a6d490 add section in changes in behavior
 -:  ------- > 24:  c069d5c update intro
 -:  ------- > 25:  3412487 Update source/news/2024/unit-1.32.0-released.rst
 -:  ------- > 26:  65b1d1f update section order
 -:  ------- > 27:  b118b15 rephrase blog mention
 -:  ------- > 28:  ac8f544 add links in github section
 -:  ------- > 29:  dbeeb2d Apply suggestions from code review
 -:  ------- > 30:  08cb947 Update source/news/2024/unit-1.32.0-released.rst
 -:  ------- > 31:  118ca67 update tense
 -:  ------- > 32:  6c7975a Update source/news/2024/unit-1.32.0-released.rst
 -:  ------- > 33:  24c51bb remove redundant sentence
 -:  ------- > 34:  5931583 Update source/news/2024/index.rst
 -:  ------- > 35:  41c6f34 remove next level
 -:  ------- > 36:  b55ea23 Update source/news/2024/unit-1.32.0-released.rst
 -:  ------- > 37:  3b0563c Update source/news/2024/unit-1.32.0-released.rst
 -:  ------- > 38:  efd4f01 add full changelog
 -:  ------- > 39:  31d7c28 update wasm mentions, capitalization
 -:  ------- > 40:  2a6e124 Update source/howto/samples.rst
 -:  ------- > 41:  c7575ce Update source/howto/samples.rst
 1:  036920b = 42:  ab6c9a5 howto/source.rst: Add info about the new --control-* options

@JTorreG JTorreG merged commit 0efd792 into nginx:master Feb 27, 2024
@ac000 ac000 deleted the ctrl-sock-perms branch February 27, 2024 18:39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

3 more reviewers

@tippexs tippexs tippexs left review comments

@lcrilly lcrilly lcrilly left review comments

@JTorreG JTorreG JTorreG left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Add Control-Socket permission Options to HowTo - Startup / Shutdown

4 participants

@ac000 @tippexs @lcrilly @JTorreG